Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use the new networkstatus flow to know if we have internet or not #1972

Merged
merged 6 commits into from
Aug 13, 2024

Conversation

tevincent
Copy link
Contributor

@tevincent tevincent commented Jul 18, 2024

depends on Infomaniak/android-core#208

no_network_flow_empty_attachment.mp4

@github-actions github-actions bot added the dependent This MR depends on another PR label Jul 18, 2024
@tevincent tevincent force-pushed the use-network-status-flow branch 2 times, most recently from 78f6c2d to 1c40c2f Compare July 19, 2024 07:10
KevinBoulongne
KevinBoulongne previously approved these changes Jul 22, 2024
@tevincent tevincent force-pushed the use-network-status-flow branch 2 times, most recently from 7ce2d72 to 45b0df3 Compare July 24, 2024 12:01
FabianDevel
FabianDevel previously approved these changes Jul 24, 2024
@tevincent tevincent force-pushed the use-network-status-flow branch 2 times, most recently from 3d069ca to aaa44e6 Compare August 6, 2024 08:40
@github-actions github-actions bot removed the dependent This MR depends on another PR label Aug 9, 2024
Copy link

github-actions bot commented Aug 9, 2024

@KevinBoulongne KevinBoulongne mentioned this pull request Aug 12, 2024
@tevincent tevincent force-pushed the use-network-status-flow branch 2 times, most recently from 090e04e to d2058e6 Compare August 12, 2024 12:25
@tevincent tevincent dismissed stale reviews from FabianDevel and KevinBoulongne August 12, 2024 12:29

Waiting for siram

@tevincent tevincent force-pushed the use-network-status-flow branch 5 times, most recently from 3d066c4 to ca3579f Compare August 12, 2024 13:42
@tevincent tevincent force-pushed the use-network-status-flow branch from ca3579f to ce06e89 Compare August 13, 2024 06:24
@sirambd sirambd requested a review from KevinBoulongne August 13, 2024 07:57
@tevincent tevincent enabled auto-merge August 13, 2024 07:57
@tevincent tevincent force-pushed the use-network-status-flow branch from 7b78a6c to 59b812b Compare August 13, 2024 08:22
@tevincent tevincent requested review from FabianDevel and removed request for FabianDevel August 13, 2024 08:28
Copy link

sonarcloud bot commented Aug 13, 2024

@tevincent tevincent merged commit 20e0b9c into master Aug 13, 2024
4 checks passed
@tevincent tevincent deleted the use-network-status-flow branch August 13, 2024 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants